CALM CLI access to CalmHub - add support for a custom API access plugin#1794
Closed
markscott-ms wants to merge 10 commits intofinos:mainfrom
Closed
CALM CLI access to CalmHub - add support for a custom API access plugin#1794markscott-ms wants to merge 10 commits intofinos:mainfrom
markscott-ms wants to merge 10 commits intofinos:mainfrom
Conversation
markscott-ms
requested review from
LeighFinegold,
Thels,
aidanm3341,
grahampacker-ms,
jpgough-ms,
lbulanti-ms,
rocketstack-matt and
willosborne
as code owners
Member
|
@markscott-ms how do you suggest we should be testing this to verify the change locally? |
Contributor
Author
|
@rocketstack-matt I can (and will) commit an example functional plugin that will work against a local dev CalmHub |
Contributor
Author
|
I was also drafting a documentation page for the website, mentioning the command line options, user config file, and environment variables. Is that desirable currently, given the discussion around what a CalmHub document id is? (Currently to work with ‘calm:’ it’s a verbose API path.) |
Contributor
Author
|
Example usage locally: 
|
6 tasks
markscott-ms
force-pushed
the
1778-calmhubauth
branch
from
ced0747 to
83e6b12
Compare
markscott-ms
force-pushed
the
1778-calmhubauth
branch
from
83e6b12 to
11d7f41
Compare
Contributor
Author
|
As discussed at #1797, get @jpgough-ms , @markscott-ms , Matt Auburn, @challamani-ms to discuss this prior to merge. |
9 tasks
25 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
As detailed in #1778 some users may not be able to access CalmHub from the CALM CLI without a custom plugin process doing the API access.
This could be due to authentication requirements, e.g. kerberos.
Type of Change
Affected Components
cli/)shared/)calm-widgets/)calm-hub/)calm-hub-ui/)docs/)calm-plugins/vscode/)Commit Message Format ✅
Testing
Checklist